Summary

Builds on #467.

Human intervention should reset both soft and hard bot turn counters. Previously the hard counter never reset, meaning a thread could be permanently locked out of bot-to-bot conversation — even with active human oversight.

Also extracts BotTurnTracker struct for testability.

Changes

Behavior fix

• Human message now resets both soft and hard counters to 0

Refactor: BotTurnTracker

Extracted inline counter logic into a testable struct:

pub(crate) struct BotTurnTracker { ... }

impl BotTurnTracker {
    pub fn new(soft_limit: u32) -> Self
    pub fn on_bot_message(&mut self, thread_id: &str) -> TurnResult
    pub fn on_human_message(&mut self, thread_id: &str)
}

enum TurnResult { Ok, SoftLimit(u32), HardLimit }

Unit tests (8 tests)

• bot_turns_increment — basic counting
• soft_limit_triggers — fires at exactly soft limit
• human_resets_both_counters — soft + hard reset, can continue
• hard_limit_triggers — fires at exactly hard limit
• hard_limit_resets_on_human — human unlocks hard-limited thread
• hard_before_soft_when_equal — when soft == hard limit, hard wins
• threads_are_independent — different threads have separate counters
• human_on_unknown_thread_is_noop — no panic on unknown thread